BundleMon

No change in files bundle size

Final result: ✅

View report in BundleMon website ➡️

Current branch size history | Target branch size history

Fais-moi rêver, on va modifier le Alerter pour qu'il affiche une Alert? Aussi quid de https://docs.cozy.io/cozy-ui/react/#/Content?id=infos ou encore https://docs.cozy.io/cozy-ui/react/#/Basics?id=banner ?

Nice :)

This PR is WIP, so no review expected yet?

Fais-moi rêver, on va modifier le Alerter pour qu'il affiche une Alert?

Modifier le Alerter je sais pas, mais en tout cas il est question ensuite d'avoir un snackbar basé sur Alert oui. J'ai pas encore poussé la réflexion jusqu'à l'Alerter mais à vu de nez comme ça, ce sera surement un nouveau composant qui va déprécier Alerter.

J'ai conscience qu'il y a aussi Infos et Banner qui sont proches, ces composants seront gérés d'une manière ou d'une autre (je sais pas encore comment) quand la PR sortira de draft

This PR is WIP, so no review expected yet?

Non, il faut également d'abord que @joel-costa valide l'aspect visuel

• Le titre doit être en gras
• Action inline : on a un bouton qui se retrouve sur 2 lignes
• Est-ce qu’on peut passer la couleur du boutton close en secondary ?
• On peut observer une variation des padding-top et bottom au passage des boutons en block/inline. Si on peut unifier ça c’est top
• ❌ Sur la variante “color”, le texte et les icones doivent passer en blanc (le nom de la variante ne me semble pas ouf, peut-être un truc genre “opaque color” ou “full color” ?)
• La variante "icon" est étrange. Tu l’as mise pour coller avec le composent ”banner” ? Moi ça me va bien mais il faudrait à mon sens aussi ajouter une variante “noIcon”, non ?
• Le container des boutons a un margin-right de -8px pour compenser le padding des text-buttons. Sauf qu’ici on a des “buttons” en small et donc avec 6px de padding, si on peut passer le container à -6px au lieu de -8px c’est top :)

Info : Pour moi le composent “info” est voué à disparaitre, remplacé par “alert”. Par contre on a “InfoCaroussel” basé sur “Info” qui gère l’affichage de plusieurs “infos” et qui ne peut pas être remplassable aujourd’hui par “alert”. Est-ce qu’on devrait ajouter une variante à “alert” qui gère du multi affichage ? Ou créer un “alertCaroussel” ?

Alerter : Pour moi le composent “alerter” devra disparaitre et être remplacé par “snackbar”, qui utilise “alert” si je ne dis pas de bêtises ?

Banner : Pour moi on a 2 choix :

• On supprime banner qu’on remplace par “alert” et on lui ajoute une variante “banner” ou “full width”
• On garde notre “banner” actuelle. qui coexiste avec “alert”. Est-ce que notre “banner” actuelle est basée sur le “alert” de MUI ? Est-ce qu’il serait pertinent de plutôt la baser sur le “alert” de Cozy UI ?

@joel-costa pour Banner à voir ensemble, tu peux triturer la doc pour te faire une idée, mais il est possible d'avoir environ le même rendu avec l'Alert actuel.

Par contre tu parles de fullwidth... l'Alert est fullwidth là, c'est pas censé être le cas ??

Sur la variante “color”, le texte et les icones doivent passer en blanc (le nom de la variante ne me semble pas ouf, peut-être un truc genre “opaque color” ou “full color” ?)

Color n'est pas une variante, c'est un exemple de la doc pour montrer qu'on peut forcer la couleur de fond si nécessaire. Attention les "variant selector" de la doc ne sont pas les variant possible sur le composant.

La variante "icon" est étrange. Tu l’as mise pour coller avec le composent ”banner” ? Moi ça me va bien mais il faudrait à mon sens aussi ajouter une variante “noIcon”, non ?

Oui exactement, c'est un exemple de doc avec une icone plus grosse.

Pourquoi une variante noIcon ? On prévoit d'avoir un Alert sans icone ? Pour l'instant ce n'est pas possible il y a toujours une icone par défaut.

Action inline : on a un bouton qui se retrouve sur 2 lignes

Dans quel cas précisément ? Sur Mobile avec beaucoup de texte et des longs boutons ? On a le comportement de Mui, donc j'imagine qu'il faut éviter ce cas de figure tout simplement et passer en "block" dans ce cas... 🤔

On peut observer une variation des padding-top et bottom au passage des boutons en block/inline. Si on peut unifier ça c’est top

Non pas sur les boutons, mais sur le container principal oui. Car c'est le cas des maquettes...

Par contre on a “InfoCaroussel” basé sur “Info” qui gère l’affichage de plusieurs “infos” et qui ne peut pas être remplassable aujourd’hui par “alert”. Est-ce qu’on devrait ajouter une variante à “alert” qui gère du multi affichage ? Ou créer un “alertCaroussel” ?

Je suis d'avis de mettre le InfoCaroussel de côté, on verra plus tard...

@joel-costa nouvelle version, normalement tout est bon concernant les précédentes remarques. J'ai ajouté des exemples dans la démo pour les filled et outlined.

Dernier souci : les boutons dans les actions d'un alert filled. Comme on utilise des boutons text, ils étaient couleur sur couleur, donc vert sur fond vert. J'ai feinté en utilisant alors des boutons primary plutôt que text, car on veut ici un bouton blanc. Ca marche plutôt bien, sauf pour le filled secondary... si t'as une idée ?

@joel-costa nouvelle version, normalement tout est bon concernant les précédentes remarques. J'ai ajouté des exemples dans la démo pour les filled et outlined.

Dernier souci : les boutons dans les actions d'un alert filled. Comme on utilise des boutons text, ils étaient couleur sur couleur, donc vert sur fond vert. J'ai feinté en utilisant alors des boutons primary plutôt que text, car on veut ici un bouton blanc. Ca marche plutôt bien, sauf pour le filled secondary... si t'as une idée ?

T'as essayé d'ajouter des boutons sur les snackbars customisés de MUI pour voir s'ils géraient le cas ? https://mui.com/material-ui/react-snackbar/#customization

Attention le lien est https://v4.mui.com/components/snackbars/#customized-snackbars (v4)

Je viens de tester, part défaut ça fait un bouton noir :

sinon en color="inherit" sur le bouton on a un bouton blanc, comme le texte

C'est un problème de nos boutons (#2194) et non de l'Alert, je pense qu'on peut passer outre ici afin d'avancer le sujet, d'autant plus qu'on peut quand même avoir un rendu convenable à mon sens en mettant la même couleur sur le bouton que sur l'alerte.

🎉 This PR is included in version 70.3.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀